Skip to content

Fix redis tasking unit tests - #8014

Open
gerrod3 wants to merge 1 commit into
pulp:mainfrom
gerrod3:rd-test-fix
Open

Fix redis tasking unit tests#8014
gerrod3 wants to merge 1 commit into
pulp:mainfrom
gerrod3:rd-test-fix

Conversation

@gerrod3

@gerrod3 gerrod3 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Assisted by: cursor-grok-4.6

Switch the fixture in the test to use redisdb fixture instead of trying to talk to the redis instance on localhost (which would fail on the CI with our rewritten host and thus silently skip the test).

Also updated the safe_release_task_locks to do nothing if the redis connection isn't there.

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

Assisted by: cursor-grok-4.6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant